Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates handleRequestInsertClipping to allow for shadowdom #326

Closed
wants to merge 1 commit into from

Conversation

willrpike
Copy link
Contributor

When we get the active element, if that element is within shadowdom, we must get the active element within the shadow dom. This fix checks to see if the shadowRoot exists as a property of the element and then updates activeElt to the activeElement in the shadow root.

Tested in firefox using both shadow dom and non shadow dom.

When we get the active element, if that element is within shadowdom, we must get the active element within the shadow dom. This fix checks to see if the shadowRoot exists as a property of the element and then updates activeElt to the activeElement in the shadow root.

Tested in firefox using both shadow dom and non shadow dom.
@willrpike
Copy link
Contributor Author

I opened #327 which uses the same code for handleRequestNewClipping as well as checking to make sure that the activeElem is a truthy value to begin with.

@willrpike willrpike closed this Feb 17, 2021
@willrpike willrpike deleted the patch-1 branch February 22, 2021 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant